Skip to content

Conversation

@Jozott00
Copy link
Collaborator

Subsystem
gRPC/Native, project setup

Problem Description
Before this, the gRPC core library was compiled from source when loading the gradle project.
With multiple targets this took at least 10 minutes.

Solution
We are now including gRPC as a pre-built static library, so it is not required to build it from source.

We also tried to use a remote repository for this, but we faced several issues.

@Jozott00 Jozott00 self-assigned this Aug 26, 2025
@Jozott00 Jozott00 added the infra Infrastructure changes (Gradle, CI, project setup, etc) label Aug 26, 2025
Signed-off-by: Johannes Zottele <[email protected]>
Signed-off-by: Johannes Zottele <[email protected]>
@Jozott00 Jozott00 requested a review from Mr3zee August 26, 2025 14:38
Copy link
Member

@Mr3zee Mr3zee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Jozott00 Jozott00 merged commit c5f0daa into grpc-common Aug 28, 2025
6 of 7 checks passed
@Jozott00 Jozott00 deleted the grpc/dep-offline branch September 2, 2025 12:26
Mr3zee pushed a commit that referenced this pull request Oct 27, 2025
…461)

* grpc-native: Build cache, not working

Signed-off-by: Johannes Zottele <[email protected]>

* grpc-native: Build cache within repository

Signed-off-by: Johannes Zottele <[email protected]>

* grpc-native: Add gRPC includes

Signed-off-by: Johannes Zottele <[email protected]>

* grpc-native: Cleanup

Signed-off-by: Johannes Zottele <[email protected]>

* grpc-native: Small fixes

Signed-off-by: Johannes Zottele <[email protected]>

* grpc-native: Use header sources from dependency instead of copy (bazel)

Signed-off-by: Johannes Zottele <[email protected]>

* grpc-native: Extract headers on demand

Signed-off-by: Johannes Zottele <[email protected]>

* grpc-native: Remove copy of include directory

Signed-off-by: Johannes Zottele <[email protected]>

* grpc-native: Remove unnecessary source dir

* grpc-native: Address PR comments

---------

Signed-off-by: Johannes Zottele <[email protected]>
Mr3zee pushed a commit that referenced this pull request Nov 12, 2025
…461)

* grpc-native: Build cache, not working

Signed-off-by: Johannes Zottele <[email protected]>

* grpc-native: Build cache within repository

Signed-off-by: Johannes Zottele <[email protected]>

* grpc-native: Add gRPC includes

Signed-off-by: Johannes Zottele <[email protected]>

* grpc-native: Cleanup

Signed-off-by: Johannes Zottele <[email protected]>

* grpc-native: Small fixes

Signed-off-by: Johannes Zottele <[email protected]>

* grpc-native: Use header sources from dependency instead of copy (bazel)

Signed-off-by: Johannes Zottele <[email protected]>

* grpc-native: Extract headers on demand

Signed-off-by: Johannes Zottele <[email protected]>

* grpc-native: Remove copy of include directory

Signed-off-by: Johannes Zottele <[email protected]>

* grpc-native: Remove unnecessary source dir

* grpc-native: Address PR comments

---------

Signed-off-by: Johannes Zottele <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

infra Infrastructure changes (Gradle, CI, project setup, etc)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants